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What we will cover 











C) A simple example of refactoring 
> Blow by blow example of changes 
> Steps for illustrated refactorings 

C) Background of refactoring 
> Where it came from Reractorinc 
> Tools wies 
> Why and When 

Q Unit testing with JUnit 
> Rhythm of development 

Q Bad Smells and their cures 








Fowler, Refactoring: Improving the Design of Existing 
Code, Addison-Wesley, 1999 














© Martin Fowler 9/11/99 








What is Refactoring 











A series of small steps, each of which 
changes the program's internal 
structure without changing its 
external behavior 


C) Verify no change in external behavior by 
> testing 
2- formal code analysis by tool 

™ |n practice good tests are essential 
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—— — — — —— Starting Class diagram 

















Movie Rental Customer 
priceCode: int m daysRented: int 






































statement() 
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Class Movie 











public class Movie ( 
public static final int CHILDRENS = 2; 
public static final int REGULAR = 0 
public static final int NEW RELEASE = 1; 


private String title; 
private int _priceCode; 


public Movie(String title, int priceCode) { 
.title = title; 
_priceCode = priceCode; 


} 


public int getPriceCode() { 
return priceCode; 
} 


public void setPriceCode(int arg) { 
.priceCode = arg; 
} 


public String getTitle () { 
return title; 
}; 
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Class Rental 











class Rental { 
private Movie _movie; 
private int _daysRented; 


public Rental(Movie movie, int daysRented) { 
_movie = movie; 
_daysRented = daysRented; 

} 

public int getDaysRented() { 
return daysRented; 

} 


public Movie getMovie() { 
return movie; 
} 
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Class Customer (almost) 











class Customer ( 
private String name; 
private Vector rentals = new Vector(); 


public Customer (String name) 
„name = name; 


hi 


public void addRental(Rental arg) { 
„rentals. addEl ement (arg); 

} 

public String getName () 
return _name; 


H 


public String statement() // see next slide 
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Customer.statement() part 1 


public String statement(] T 
double total Amount = 0; 
int frequentRenterPoints - 0; 
Enumeration rentals = rentals.elements(); 
String result = "Rental Record for " + getName() + "in"; 
while (rentals.hasMoreElements()) ( 
double thisAmount = 0; 
Rental each = (Rental) rentals.nextEl ement(); 








lidetermine amounts for each line 
switch (each.getMovie().getPriceCode()) { 
case Movie. REGULAR: 
thisAmount += 2; 
if (each. getDaysRented() > 2) 
thisAmount += (each. getDaysRented() - 2) * 1.5; 
break; 
case Movie. NEW RELEASE: 
thisAmount += each. getDaysRented() * 3; 
break; 
case Movie. CHILDRENS: 
thisAmount += 1.5; 
if (each. getDaysRented() > 3) 
thisAmount += (each. getDaysRented() - 3) * 1.5; 
break; 


continues on next slide 
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Customer.statement() part 2 











1l add frequent renter points 

frequentRenterPoints ++; 

1| add bonus for a two day new release rental 

if ((each.getMovie().getPriceCode() == Movie. NEW RELEASE) && 
each.getDaysRented() > 1) frequentRenterPoints ++; 


//show figures for this rental 

result += "|t" + each. getMovie().getTitle()+ "At" + 
String. valueOf(thisAmount) + "\n"; 

totalAmount += thisAmount; 


} 

lladd footer lines 

result += "Amount owed is " + String. valueOf(total Amount) + "\n"; 

result += "You earned " + String. valueOf(frequentRenterPoints) + 
" frequent renter points"; 

return result; 
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Interactions for statement 














aR ental 


























statement 











* [for all rentals] 








getMovie 





getP riceCode 





getDaysRented 
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Sample Output 











Rental Record for Dinsdale Pirhana 
Monty Python and the Holy Grail 
Ran2 
Star Trek 27 6 
Star Wars 3.2 3 
Wallace and Gromit 6 

Amount owed is 20.5 

You earned 6 frequent renter points 
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Requirements Changes 








Q Produce an html version of the statement 
Q The movie classifications will soon 
change 


> together with the rules for charging and for 
frequent renter points 
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Extract Method 











You have a code fragment that can be grouped together 
Turn the fragment into a method whose name explains the 
purpose of the method. 


void printOwing() ( 
printBanner(); 


IIprint details 
System out.println ("name:* + name); 
System out. printin ("amount® + getOutstanding()); 


void printOwing() { 
print Banner(); 
printDetails(getOutstanding()); 
} 


void printDetails (double outstanding) ( 
System out.println ("name:* + name); 
System out. printin ("amount® + outstanding); 
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Candidate Extraction 











public String statement() { 

double totalAmount = 0; 
int frequentRenterPoints = 0; 
Enumeration rentals = rentals.elements(); 
String result = "Rental Record for " + getName() + "in"; 
while (rentals. hasMoreElements()) { 

double thisAmount = 0; 

Rental each = (Rental) rentals. nextEl ement(); 


|| determine amounts for each line 
switch (each.getMovie().getPriceCode()) { 
case Movie. REGULAR: 
thisAmount += 2; 
if (each. getDaysRented() > 2) 
thisAmount += (each. getDaysRented() - 2) * 1.5; 
break; 
case Movie. NEW RELEASE: 
thisAmount += each. getDaysRented() * 3; 
break; 
case Movie. CHILDRENS: 
thisAmount += 1.5; 
if (each. getDaysRented() > 3) 
thisAmount += (each. getDaysRented() - 3) * 1.5; 
break; 
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Steps for Extract Method 











Create method named after intention of 
code 


Copy extracted code 


Look for local variables and parameters 
> turn into parameter 
2 turn into return value 
2 declare within method 
Compile 
Replace code fragment with call to new 
method 


Compile and test 
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Extracting the Amount 
Calculation 











private int amountFor(Rental each) ( 
int thisAmount = 0; 
switch (each. getMovie().getPriceCode()) { 
case Movie. REGULAR: 
thisAmount += 2; 
if (each. getDaysRented() > 2) 
thisAmount += (each. getDaysRented() - 2) * 1.5; 
break; 
case Movie. NEW RELEASE: 
thisAmount += each. getDaysRented() * 3; 
break; 
case Movie. CHILDRENS: 
thisAmount += 1.5; 
if (each. getDaysRented() > 3) 
thisAmount += (each. getDaysRented() - 3) * 1.5; 
break; 


return thisAmount; 


} 
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Statement() after extraction 


public String statement() ( 
double total Amount = 0; 
int frequentRenterPoints - 0; 
Enumeration rentals = rentals.elements(); 
String result = "Rental Record for " + getName() + "in"; 
while (rentals.hasMoreElements()) ( 
double thisAmount = 0; 
Rental each = (Rental) rentals.nextEl ement(); 











thisAmount - amountFor(each); 


1l add frequent renter points 

frequentRenterPoints ++; 

1| add bonus for a two day new release rental 

if ((each.getMovie().getPriceCode() == Movie. NEW RELEASE) && 
each.getDaysRented(] > 1) frequentRenterPoints ++; 


IIshow figures for this rental 

result += "At" + each. getMovie().getTitle()+ "At" + 
String. valueOf(thisAmount) + "\n"; 

totalAmount += thisAmount; 


} 

lladd footer lines 

result += "Amount owed is " + String.valueOf(total Amount) + "\n"; 

result += "You earned " + String.valueOf(frequentRenterPoints) + 
" frequent renter points"; 

return result; 
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Extracting the amount 
calculation (2) 











private double amountFor(Rental each) ( 
double thisAmount = 0; 
switch (each.getMovie().getPriceCode()) ( 
case Movi e. REGULAR: 
thisAmount += 2; 
if (each.getDaysRented() » 2) 
thisAmount += (each.getDaysRented() - 2) * 1.5; 
break; 
case Movie. NEW RELEASE: 
thisAmount += each.getDaysRented() * 3; 
break; 
case Movi e. CHI LDRENS: 
thisAmount += 1.5; 
if (each.getDaysRented() » 3) 
thisAmount += (each.getDaysRented() - 3) * 1.5; 
break; 
} 


return thisAmount; 
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Change names of variables 








private double amountFor(Rental aRental) ( 
double result = 0; 
switch (aRental.getMovie().getPriceCode()) { 
case Movie. REGULAR: 
result += 2; 
if (aRental.getDaysRented() > 2) 
result += (aRental.getDaysRented() - 2) * 1.5; 
break; 
case Movie. NEW RELEASE: 
result += aRental.getDaysRented() * 3; 
break; 
case Movie. CHILDRENS: 
result += 1.5; 
if (aRental.getDaysRented() > 3) 
result += (aRental.getDaysRented() - 3) * 1.5; 
break; 
} 


return result; 


Is this important? 
Is this method in the right place? 
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Move Method 











A method is, or will be, using or used by more features of 
another class than the class it is defined on. 
Create a new method with a similar body in the class it uses 
most. Either turn the old method into a simple delegation, or 
remove it altogether. 








Class 1 














aMethod() 














Class 2 Class 2 














aMethod() 
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1! 








Steps for Move method 








Declare method in target class 
Copy and fit code 


Set up a reference from the source object 
to the target 


Turn the original method into a 
delegating method 


>  amountOf(Rental each) {return each. charge() } 
> Check for overriding methods 


Compile and test 


Find all users of the method 
> Adjust them to call method on target 


Remove original method 
Compile and test 
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Moving amount() to Rental 











class Rental 
double getCharge() { 
double result = 0; 
switch (getMovie().getPriceCode()) { 
case Movie. REGULAR: 
result += 2; 
if (getDaysRented() > 2) 
result += (getDaysRented() - 
break; 
case Movie. NEW RELEASE: 
result += getDaysRented() * 3; 
break; 
case Movie. CHILDRENS: 
result += 1.5; 
if (getDaysRented() » 3) 
result += (getDaysRented() - 
break; 
) 
return result; 


} 








Rental Customer 











Movie 





daysR ented: int 


priceCode: int iChargel statement() 
getCharge 
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Altered statement 











class Customer... 
public String statement() ( 
double total Amount = 0; 
int frequentRenterPoints - 0; 
Enumeration rentals = rentals.elements(); 
String result = "Rental Record for " + getName() + "in"; 
while (rentals. hasMoreElements()) { 
double thisAmount = 0; 
Rental each = (Rental) rentals. nextElement(); 


thisAmount = each. getCharge(); 

1] add frequent renter points 

frequentRenterPoi nts ++; 

1| add bonus for a two day new release rental 

if ((each.getMovie(].getPriceCode() == Movie. NEW RELEASE) 
each.getDaysRented() > 1) frequentRenterPoints ++; 


/ishow figures for this rental 

result += "At" + each.getMovie().getTitle()* "1t" + 
String.valueOf(thisAmount) + "in"; 

totalAmount += thisAmount; 


} 

lladd footer lines 

result += "Amount owed is " + String. valueOf(total Amount) + "\n"; 
result += "You earned " + String. valueOf(frequentRenterPoints) + 
" frequent renter points"; 

return result; 
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Problems with temps 











class Customer... 
public String statement() ( 
double total Amount = 0; 
int frequentRenterPoints - 0; 
Enumeration rentals = rentals.elements(); 
String result = "Rental Record for " + getName() + "in"; 
while (rentals.hasMoreElements()) ( 
double thisAmount = 0; 
Rental each = (Rental) rentals.nextElement(); 


thisAmount = each. getCharge(); 

1l add frequent renter points 

frequentRenterPoints ++; 

1| add bonus for a two day new release rental 

if ((each.getMovie(].getPriceCode() == Movie. NEW RELEASE) && 
each.getDaysRented() > 1) frequentRenterPoints ++; 


//show figures for this rental 

result += "At" + each. getMovie().getTitle()+ "\t" + 
String. valueOf(thisAmount) + "in"; 

totalAmount += thisAmount; 


} 

lladd footer lines 

result += "Amount owed is " + String. valueOf(total Amount) + "\n"; 
result += "You earned " + String. valueOf(frequentRenterPoints) + 
" frequent renter points"; 

return result; 
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A Word About Performance 














The best way to optimize 
performance is to first write a well 
factored program, then optimize it. 


Most of a program's time is taken in a 
small part of the code 


Profile a running program to find these 
"hot spots" 
> You won't be able to find them by eye 


O ptimize the hot spots, and measure the 
improvement 





McConnell Steve, Code Complete: A Practical Handbook of 
Software Construction, Microsoft Press, 1993 
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Replace Temp with Query 








You are using a temporary variable to hold the result of an 
expression. 
Extract the expression into a method. Replace all references 
to the temp with the expression. The new method can then 
be used in other methods. 


double basePrice = quantity * itemPrice; 
if (basePrice » 1000) 

return basePrice * 0.95; 
else 

return basePrice * 0.98; 


if (basePrice(] » 1000) 
return basePrice() * 0.95; 
else 
return basePrice() * 0.98; 


double basePrice() { 
return quantity * itemPrice; 


} 
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— Steps for Replace temp with Query 








Find temp with a single assignment 
Extract Right Hand Side of assignment 


Replace all references of temp with new 
method 


Remove declaration and assignment of 
temp 


Compile and test 
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thisAmount removed 











public String statement() ( 
double total Amount = 0; 
int frequentRenterPoints - 0; 
Enumeration rentals = rentals.elements(); 
String result = "Rental Record for " + getName() + "\n"; 
while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals. nextEl ement(); 


1l add frequent renter points 

frequentRenterPoints ++; 

1| add bonus for a two day new release rental 

if ((each.getMovie().getPriceCode() == Movie. NEW RELEASE) && 
each.getDaysRented() > 1) frequentRenterPoints ++; 


//show figures for this rental 

result += "At" + each. getMovie().getTitle()+ "At" + 
String. valueOf(each.getCharge()) + "in"; 

total Amount += each. getCharge(); 


} 

lladd footer lines 

result += "Amount owed is " + String.valueOf(total Amount) + "\n"; 

result += "You earned " + String.valueOf(frequentRenterPoi nts) + 
" frequent renter points"; 

return result; 
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class Customer... 
public String stat 





Extract and move 
frequentRenterPoints() 











ement() ( 


double total Amount = 0; 
int frequentRenterPoints = 0; 
Enumeration rentals = _rentals.elements(); 


String result 


= "Rental Record for " + getName() + "\n"; 


while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals. nextEl ement(); 
frequentRenterPoints += each. get Frequent Renter Points(); 


//show figures for this rental 


result += 


"At" + each. getMovie().getTitle()+ "\t" + 
String. valueOf(each.getCharge()) + "\n"; 


total Amount += each. getCharge(); 


} 


lladd footer | 


Ines 


result += "Amount owed is " + String.valueOf(total Amount) + "\n"; 
result += "You earned " + String. valueOf(frequentRenterPoints) + 
" frequent renter points"; 


return result; 





Movie 
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After moving charge and 
frequent renter points 














Rental 








Customer 





daysR ented: int 











priceC ode: int 











getCharge() statement() 
getF requentR enterP oints() 















































statement 











* [for all rentals] 








getCharge 





getP riceCode 








getF requentR enterP oints 























getP riceCode 
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More temps to kill 











class Customer... 
public String statement() ( 

double total Amount = 0; 

int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = "Rental Record for " + getName() + "\n"; 

while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals. nextEl ement(); 
frequentRenterPoints += each. getFrequent Renter Points(); 


//show figures for this rental 

result += "At" + each. getMovie().getTitle()+ "At" + 
String. valueOf(each. getCharge()) + "\n"; 

total Amount += each. getCharge(); 


lladd footer lines 

result += "Amount owed is “ + String.valueOf(total Amount) + "\n"; 

result += "You earned " + String. valueOf(frequentRenterPoints) + 
" frequent renter points"; 

return result; 


© Martin Fowler 9/11/99 





The new methods 











class Customer... 


private double getTotalCharge() { 
double result = 0; 
Enumeration rentals = _rentals.elements(); 
while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals. nextEl ement(); 
result += each. getCharge(); 


return result; 


private int getTotal Frequent Renter Points() ( 
int result = 0; 
Enumeration rentals = _rentals.elements(); 
while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals. nextEl ement(); 
result += each. getFrequentRenterPoints(); 
} 


return result; 
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1! 








The temps removed 








public String statement() ( 
Enumeration rentals = rentals.elements(); 
String result = "Rental Record for " + getName() + "in"; 
while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals. nextEl ement(); 


//show figures for this rental 
result += "\t" + each. getMovie().getTitle()+ "At" + 


String. valueOf(each. getCharge()) + "\n"; 
} 


lladd footer lines 


result += "Amount owed is " + String. valueOf(getTotal Charge()) + "in"; 


result += "You earned " + String. valueOf(getTotal FrequentRenterPoints()) + 
" frequent renter points"; 
return result; 
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After replacing the totals 











Rental Customer 





Movie daysR ented: int 











statement() 
getTotalC harge() 
getT otalF requentR enter? oints() 


priceCode: int getCharge() 
getF requentR enter? oints() 





























aCustomer aRental aMovie 


























statement 





We ae 
getTotalCharge 





* [for all rentals] getCharge 








getP riceCode 




















1 
getT otalF requentR enter? oi 
El 


* [for all rentals] getF requentR enterP oints 





getP riceCode 
> 
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html Statement() 











public String htmlStatement() { 
Enumeration rentals = rentals.elements(); 
String result = "«Hl»Rentals for <EM>" + getName() + "</EM></H1><P>\n"; 
while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals. nextEl ement(); 
//show figures for each rental 
result += each. getMovie().getTitle()+ ": " + 
String. valueOf(each. getCharge()) + "<BR>\n"; 
} 
lladd footer lines 
result += "<P>You owe <EM>" + String. valueOf(getTotalCharge()) + 
"<| EM><P>\ n"; 
result += "On this rental you earned <EM>" + 
String. valueOf(getTotal FrequentRenterPoints()) + 
"</EM> frequent renter points<P>"; 
return result; 
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— The current getCharge method 











class Rental 
double getCharge() { 
double result = 0; 
switch (getMovie().getPriceCode()) { 
case Movie. REGULAR: 
result += 2; 
if (getDaysRented() > 2) 
result += (getDaysRented() - 2) * 1.5; 
break; 
case Movie. NEW RELEASE: 
result += getDaysRented() * 3; 
break; 
case Movie. CHILDRENS: 
result += 1.5; 
if (getDaysRented() > 3) 
result += (getDaysRented() - 3) * 1.5; 
break; 
} 


return result; 
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getCharge moved to Movie 








class Rental... 
double getCharge() { 
return movie.getCharge( daysRented); 


class Movie .. 
double getCharge(int daysRented) ( 
double result = 0; 
switch (getPriceCode()) ( 
case Movi e. REGULAR: 
result += 2; 
if (daysRented > 2) 
result += (daysRented - 
break; 
case Movie. NEW RELEASE: 
result += daysRented * 3; 
break; 
case Movie. CHILDRENS: 
result += 1.5; 
if (daysRented » 3) 
result += (daysRented - 
break; 
} 
return result; 


} 


ü Do the same with frequentRenterPoints() 
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Consider inheritance 














Movie 








getC harge 


























Regular Movie Childrens Movie New Release Movie 








getC harge getC harge getC harge 























How's this? 
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Using the State Pattern 

















Movie Price 























getCharge © getCharge 




















return price.getCharge 




















Regular Price Childrens Price New Release Price 











getCharge getCharge getCharge 
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t Replace Type Code with State/Strategy 





You have a type code which affects the behavior of a class 
but you cannot use subclassing. 
Replace the type code with a state object. 





Employee 





ENGINEER : int 
SALESMAN : int 
type : int 


V 


Employee Employee Type 


















































Engineer Salesman 
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— Steps for Replace Type Code with 
State/Strategy 
Create a new state class for the type code 


Add subclasses of the state object, one 
for each type code. 


Create an abstract query in the superclass 
to return the type code. Override in 
subclasses to return correct type code 


Compile 


Create field in old class for the state 
object. 


Change the type code query to delegate 
to the state object. 


Change the type code setting methods to 
assign an instance of the subclass. 


Compile and test. 
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— Price codes on the price hierarchy 











abstract class Price { 
abstract int getPriceCode(); 
} 
class ChildrensPrice extends Price { 
int getPriceCode() { 
return Movie. CHILDRENS; 
} 
} 
class NewReleasePrice extends Price { 
int getPriceCode() { 
return Movie. NEW RELEASE; 
} 


class RegularPrice extends Price { 
int getPriceCode() { 
return Movie. REGULAR; 
} 
} 
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Change accessors on Movie 











public int getPriceCode() ( 
return priceCode; 

} 

public setPriceCode (int arg) { 
.priceCode = arg; 

} 


private int _priceCode; 


* 


public int getPriceCode() { 
return price.getPriceCode(); 
} 
public void setPriceCode(int arg) { 
switch (arg) { 
case REGULAR: 
_price = new RegularPrice(); 
break; 
case CHILDRENS: 
_price = new ChildrensPrice(); 
break; 
case NEW RELEASE: 
_price = new NewReleasePrice(); 
break; 
default: 
throw new Illegal ArgumentException("Incorrect Price Code"); 
} 
} 


private Price _price; 
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—— Replace Conditional With Polymorphsim 








You have a conditional that chooses different behavior 
depending on the type of an object 
Move each leg of the conditional to an overriding method 
in a subclass. Make the original method abstract 


double getSpeed() { 
switch ( type] { 
case EUROPEAN: 
return getBaseSpeed|); 
case AFRICAN: 
return getBaseSpeed() - getLoadFactor() * number OfCoconuts; 
case NORWEI GI AN BLUE: 
return ( isNailed) ? 0: getBaseSpeed( voltage]; 
} 


throw new Runti meException ("Should be unreachable"); 





Bird 





getS peed 




















European African Norweigian Blue 




















getS peed getS peed getS peed 
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— Steps for Replace Conditional with 
Polymorphism 











Move switch to superclass of inheritance 
structure 


Copy one leg of case statement into 
subclass 


Compile and test 
Repeat for all other legs 


Replace case statement with abstract 
method 
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Move getCharge to Price 








class Movie.. 

double getCharge(int daysRented) ( 
return price.getCharge(daysRented); 

) 


class Price. 
double getCharge(int daysRented) ( 
double result = 0; 
switch (getPriceCode()) ( 
case Movi e. REGULAR: 
result += 2; 
if (daysRented > 2) 
result += (daysRented - 
break; 
case Movie. NEW RELEASE: 
result += daysRented * 3; 
break; 
case Movie. CHILDRENS: 
result += 1.5; 
if (daysRented » 3) 
result *- (daysRented - 
break; 
} 


return result; 
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Override getCharge in the 
subclasses 











Class RegularPrice.. 
double getCharge(int daysRented) { 
double result = 2; 
if (daysRented > 2) 
result += (daysRented - 2) * 1.5; 
return result; 
} 
Class ChildrensPrice 
double getCharge(int daysRented) { 
double result = 1.5; 
if (daysRented > 3) 
result += (daysRented - 3) * 1.5; 
return result; 


} 


Class NewReleasePrice.. 
double getCharge(int daysRented) { 
return daysRented * 3; 


Q Do each leg one at a time 
Q then... 





Class Price. 
abstract double getCharge(int daysRented); 
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Similar Statement Methods 











public String statement() ( 
Enumeration rentals = rentals.elements(); 
String result = "Rental Record for " + getName() + "\n"; 
while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals. nextElement(); 
result += "\t" + each. getMovie().getTitle()+ "At" + 
String. valueOf(each. getCharge()) + "in"; 
} 
result += "Amount owed is " + String. valueOf(getTotalCharge()) + "\n"; 
result += "You earned " + String. valueOf(getTotal FrequentRenterPoints()) + 
" frequent renter points"; 
return result; 


public String html Statement() { 

Enumeration rentals = rentals.elements(); 

String result = "«Hl»Rentals for <EM>" + getName() + "</EM></H1><P>\n"; 

while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals.nextElement(); 
result += each.getMovie().getTitle()* ": " + 
String. valueOf(each.getCharge()) + "<BR>\n"; 

} 

result += "«P»You owe <EM>" + 
String. valueOf(getTotalCharge()) + "</EM><P>\n"; 

result += "On this rental you earned <EM>" + 
String. valueOf(getTotalFrequentRenterPoints()) + 
"</EM> frequent renter points<P>"; 

return result; 
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Form Template Method 








You have two methods in subclasses that carry out similar 
steps in the same order, yet the steps are different 
Give each step into methods with the same signature, so that 
the original methods become the same. Then you can pull 


them up. 





























Residential Site 


Lifeline Site 














getBillableAmounto 








getBillableAmount> 





> 





Site 











getBillableAmount ? 
getBaseAmount | 
getTaxAmount | 























Residential Site 


LifelineSite 











getBaseAmount 
getTaxAmount 





getBaseAmount 
getTaxAmount 

















double base - units * rate * 0.5; 
double tax = base * Site. TAX RATE * 0.2; 
return base + tax; 

















return getBaseAmount() + getTaxAmount(); 























return base + tax; 


double base - units * rate; 
double tax = base * Site. TAX RATE; 
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— Steps for Form Template Method 











() Take two methods with similar overall 
structure but varying pieces 
> Use subclasses of current class, or create a 


strategy and move the methods to the 
strategy 





At each point of variation extract methods 
from each source with the the same 
signature but different body. 


Declare signature of extracted method in 


superclass and place varying bodies in 
subclasses 


When all points of variation have been 
removed, move one source method to 
superclass and remove the other. 
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Create a Statement Strategy 











class Customer 
public String statement() ( 
return new TextStatement(). val ue(this); 


} 


class TextStatement { 
public String value(Customer aCustomer) { 
Enumeration rentals = aCustomer.getRentals(); 
String result = "Rental Record for " + aCustomer.getName() + "\n"; 
while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals. nextEl ement(); 
result += "At" + each. getMovie().getTitle()+ "lt" + 
String. valueOf(each.getCharge()) + "in*; 
} 
result += "Amount owed is " + 
String.valueOf(aCustomer.getTotalCharge()) + "\n'"; 
result += "You earned " + 
String. valueOf(aCustomer.getTotalFrequentRenterPoints()) + 
" frequent renter points"; 
return result; 


Q Do the same with htmlStatement() 
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Extract Differences 











class TextStatement... 
public String value(Customer aCustomer) ( 
Enumeration rentals = aCustomer.getRentals(); 
String result = headerString(aCust omer); 
while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals. nextEl ement(); 
result += "At" + each. getMovie().getTitle()+ "lt" + 
String. valueOf(each.getCharge()) + "in"; 
} 
result += “Amount owed is " + 
String. valueOf(aCustomer.getTotalCharge()) + "\n'"; 
result += "You earned " + 
String. valueOf(aCustomer.getTotalFrequentRenterPoints()) + 
" frequent renter points"; 
return result; 


String headerString(Customer aCustomer) { 
return "Rental Record for " + aCustomer. getName() + "in"; 


} 


C) Do the same with htmlStatement 


class Html Statement... 
String headerString(Customer aCustomer) { 
return "«Hl»Rentals for <EM>" + aCustomer.getName() + "</EM></H1><P>\n"; 


} 
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Continue extracting 
class TextStatement 


public String value(Customer aCustomer) { 

Enumeration rentals = aCustomer.getRentals(); 

String result = headerString(aCustomer); 

while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals.nextElement(); 
result += eachRental String(each); 

} 

result += footerString(aCustomer); 

return result; 


String eachRentalString (Rental aRental) { 
return "At" + aRental.getMovie().getTitle()+ "\t" + 
String. valueOf(aRental.getCharge()) + "\n"; 


String footerString (Customer aCustomer) { 
return "Amount owed is " + 
String. valueOf(aCustomer.getTotalCharge()) + "in" + 
"You earned " + 
String.valueOf(aCustomer.getTotalFrequentRenterPoints()) + 
" frequent renter points"; 


C] Do the same with htmlStatement 
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Pull up the value method 








class Statement... 
public String value(Customer aCustomer) { 
Enumeration rentals = aCustomer. getRentals(); 
String result = headerString(aCustomer); 
while (rentals. hasMoreElements()) { 
Rental each = (Rental) rentals.nextElement(); 
result += eachRental String( each); 
} 
result += footerString(aCustomer); 
return result; 


abstract String headerString(Customer aCustomer); 
abstract String eachRental String (Rental aRental); 
abstract String footerString (Customer aCustomer); 
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State of Classes 














Statement 





Customer 











value(Customer) 

headerS tring(C ustomer) 
eachR ental tring(R ental) 
footerS tring(C ustomer) 








statement() 
htmlStatement() 



































Html Statement 


Text Statement 











headerString(Customer) 
eachRentalString(Rental) 
footerString(C ustomer) 








headerString(Customer) 
eachRentalString(Rental) 
footerString(C ustomer) 
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In this example 








C] We saw a poorly factored program 
improved 
> easier to add new services on customer 
> easier to add new types of movie 
C) No debugging during refactoring 
> appropriate steps reduce chance of bugs 
> small steps make bugs easy to find 
C] Illustrated several refactorings 
2- Extract Method 
Move Method 
Replace Temp with Query 
Replace Type Code with State/Strategy 
Replace Switch with Polymorphism 
Form Template Method 
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Definitions of Refactoring 








Q Loose Usage 
> Reorganize a program (or something) 
Q As a noun 


> a change made to the internal structure of 
some software to make it easier to 
understand and cheaper to modify, without 
changing the observable behavior of that 
software 


ū Asa verb 


> the activity of restructuring software by 
applying a series of refactorings without 
changing the observable behavior of that 
software. 
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— Where Refactoring Came From 











Q Ward Cunningham and Kent Beck 
> Smalltalk style 
Q Ralph Johnson at University of Illinois at 
Urbana-Champaign 
Q Bill O pdyke's Thesis 


://st.cs.uiuc.edu/pub/papers/refactoring/opdyke-thesis.ps.Z 


Q John Brant and Don Roberts: The 
Refactoring Browser 
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Refactoring Tools 











ü Based on provable transformations 
> Build parse tree of programs 


> Mathematic proof that refactoring does not 
change semantics 


> Embed refactoring in tool 
(] Speeds up refactoring 


> Extract method: select code, type in method 
name. 


> No need for tests (unless dynamic reflection) 
> Big speed improvement 

QO) Not Science Fiction 
> Available for Smalltalk 
http://st-www.cs.uiuc.edu/~ brant/RefactoringBrowser 
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The Importance of Tests 











Even with a tool, testing is important 
> Notall refactorings can be proven 
Write tests as you write the code 
Make the test self-checking 
> return “OK” if good, errors if not 
Run a suite with a single command 
Test with every compile 





è ftp://www.armaties.com/D/home/ 
armaties/ftp/TestingFramework/ 


è http://ourworld.compuserve.com/ 
homepages/Martin Fowler 
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The Two Hats 











S P. 


Adding Function Refactoring 


Q Add new capabilities to Does not add any new 
the system features 

Q Adds new tests Does not add tests (but 

Q Get the test working may change some) 

Restructure the code to 

remove redundancy 











Swap frequently between the hats, but 
only wear one at a time 
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Why Refactor 








QO) To improve the software design 
> combat’s “bit rot" 
> makes the program easier to change 
Q To make the software easier to 
understand 
> write for people, not the compiler 
> understand unfamiliar code 
C] To help find bugs 
> refactor while debugging to clarify the code 


Refactoring helps you program faster! 
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When should you refactor? 











Q The Rule of Three 
Q To add new functionality 
> refactor existing code until you understand it 
> refactor the design to make it easy to add 
Q To find bugs 
> refactor to understand the code 
Q For code reviews 
> immediate effect of code review 
> allows for higher level suggestions 








Don't set aside time for refactoring, include it 
in your normal activities 
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— What do you tell your manager 











Dont! 


Q) If the manager is really concerned about 
quality 
> then stress the quality aspects 
C] Otherwise you need to develop as fast as 
possible 


> you're the professional, so you know to do 
what makes you go faster 
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Problems with Refactoring 








Q We don’t know what they are yet 


Q Database Migration 


> Insulate persistent database structure from 
your objects 


> With OO databases, migrate frequently 
Q) Published Interfaces 

2 Publish only when you need to 

> Don’t publish within a development team 
Q Without working tests 

> Don't bother 
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Design Decisions 








CL) In the moment 
> Consider current needs 
> Patch code when new needs appear 


C) Up front design 


> Consider current needs and possible future 
needs 


> Design to minimize change with future needs 
> Patch code if unforeseen need appears 
Q Evolutionary design 


> Consider current needs and possible future 
needs 


> Trade off cost of current flexibility versus cost 
of later refactoring 


> Refactor as changes appear 
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Extreme Programming 








P 


Q Methodology developed by Kent Beck 
Q Designed to adapt to changes 
Q Key Practices 
> Iterative Development 
> Self Testing Code 
> Refactoring 
> Pair Programming 
C) Moves away from up-front design 











http://www.armaties.com/extreme.htm 
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Team Techniques 











C) Encourage refactoring culture 
> nobody gets things right first time 
> nobody can write clear code without reviews 
> refactoring is forward progress 
C] Provide sound testing base 
> tests are essential for refactoring 
> build system and run tests daily 
Q) Pair Programming 


> two programmers working together can be 
quicker than working separately 


> refactor with the class writer and a class user 
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3: 





Creating Your Own Refactorings 











Consider a change to a program 


Should it change the external behavior of 
the system 


Break down the change into small steps 
> Look for points where you can compile and 
test 








Carry out the change, note what you do 
> Ifa problem occurs, consider how to 
eliminate it in future 
Carry it out again, follow and refine the 
notes 
After two or three times you have a 
workable refactoring 
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Self Testing Code 











Build and run tests as you build production 
code 


ü For each piece of new function 
> Write the test 
> Write the production code 
> Run your test suite 
> Ifit works you're done 
Q Developers 


> Do this with every small bit of function you 
add 


Q QAor Test Group 
> Do this with each increment 
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The JUnit Framework 








Simple, but effective framework for 
collecting and running unit tests in Java 


Written by Erich Gamma and Kent Beck 
> based on Kent's framework for Smalltalk 


Easily define tests 
Easily group tests into suites 
Easily run suites and monitor results 





ftp://www.armaties.com/D/home/armaties/ftp/ 
TestingFramework/JU nit/ 
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An Example Coding Session 








C) Build a Money class 
> combines amount and currency 
> provides arithmetic operations 
> use of Quantity pattern 


C] Build a MoneyTester class 


Fowler, Martin. Analysis Patterns: Reusable O bject 
Models, Addison Wesley 1997 
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Fitting into the Framework 














testframework 











«interface» 























TestSuite TestCase 












































MoneyTester 
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The Junit GUI 











Ka Run Test Suite -joj x| 


Enter the name ofthe TestCase class: 


| | .suite( ) 


Progress: 


| 


Errors and Failures: 





Runs: 0 Errors: 0 Failures: 0 


Shay | 
_ S| Quit | 
Q There is also a text console interface 


Q Invoke with 


> java test.textui.TestRunner MoneyTester 
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Creating MoneyTester 








import test.framework. *; 
public class MoneyTester extends TestCase{ 


public MoneyTester(String name) ( 
Super(name); 


public static Test suite() { 
return new TestSui te(MoneyTester.class); 
} 
} 
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The First Test 











MoneyTester 
public void testPrint() { 
Money d15 = new Money(15, "USD"); 
assertEquals("15 USD", d15.toString()); 
} 


public class Money { 
private long _amountI nPennies; 
private String _currencyCode; 


public Money(double amount, String currencyCode) { 
_amountinPennies = Math.round (amount * 100); 
_CurrencyCode = currencyCode; 


} 
public String toString() { 
return ("" + amounti nPennies + " " + currencyCode); 


} 


} 
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First Test Result 











Ener the name of the TesiCase dass: 


[ money MonerTester sume) Run | 


Progress: 


Runs 4 Errors O Failures: 4 


Errors and Falures 


p money MorryTa ster tastPrint expactad 17 USD but was "1500 USD* ; | 
[ Finished: 0.45 seconds Quit | 
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Fixing the First Test 








public void testPrint() ( 
Money d15 = new Money(15, "USD"); 
assertEquals("15 USD", dl5.toString()); 
} 


public class Money { 
private long amountl nPenni es; 
private String _currencyCode; 


public Money(double amount, String currencyCode) { 
.amountinPennies = Math. round (amount * 100); 
_CurrencyCode = currencyCode; 


} 
public String toString() { 


return ("" + getAmount() +" " + _currencyCode); 


} 
private double getAmount() { 

return (double) amounti nPennies / 100; 
} 


} 
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After the fix 











K Run Test Suite 
Errar the name ofthe Tastlace class 


| money.MonayTaster E 


Progiess 


Runs 1 Errors: O Failures. 4 
Errors and Failures 


Failure: money. MoneyT aster testPrint'mpactsd*1 5 VSD ku 4as"15.0 USD" 


| Finished: 0.21 serands Quit | 
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Changing the Test 








public void testPrint() { 
Money d15 = new Money(15, "USD"); 


assertEquals("15.0 USD", dl15.toString()); 
} 


Enter fe nime offie Testcase rises 


fane Nona ester tec) | Run J 
Progress 


pe ————— 


Rune 4 Eno O Faluas o 
Enora enit Faitures 


Aeg 


[ Finiered 0 4 ratoris Qut | 





QO) Don't consider fancy formatting yet 
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Checking Rounding 











public void testRound() { 
Money dRounded = new Money (1.2350, "USD") 
assertEquals ("1.24 USD", dRounded.toString()); 


Enfartne nome aftha TachCase chass 
money. MoneyTester 


Progress: 


| 


Enors and Fatures: 





Munt 2 Errors U Fotues u 


[ Finished: 0.11 seconds Qut 
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Adding Addition 








C] Add two monies together in the same 
currency 


public void testAddition() { 
Money d15 = new Money (15, "USD") 
Money d2 51 = new Money (2.51, "USD"); 
assertEquals (new Money (17.51, "USD"), 
dl5.plus(d2 51)); 
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Need to add equals 











C) Money is a value, so needs a special 
equals (and hash) 


public void testEquals() { 
Money d2 51a new Money (2.51, "USD"); 
Money d2 51b new Money (2.51, "USD"); 


assertEquals (d2 51a, d2 51b); 


} 


Ea Run Teast Suite 
Erter the nome ofthe TertCaee rises 
morery Money Tester 


Progress 


Ru 3 Grom Q Falues 1*4 


Eres and Falurer 


[ Fintashwd 227 seconds E | 
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The Equals Method 








Money 
public boolean equals (Object arg) { 
if (! (arg instanceof Money)) return false; 
Money moneyArg = (Money) arg; 
return ( amountinPennies == moneyArg. amountinPennies && 
_currencyCode. equals(moneyArg. currencyCode]); 


Erme ssec e re Tene ane 
[eee Wane, et ane EF] 


Pragete 





Mm JP Fs 5 tae v 
Enos wd faune 


Ts 


zoco oe 
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Additional Tests 











public void testCloseNumbersNotEqual() { 
Money d2 51a - new Money (2.515, "USD") 
Money d2 51b = new Money (2.5149, "USD"); 
assert(! d2 51a.equals(d2 51b)); 


} 

public void testDifferentCurrencyNotEqual() { 
Money d2 51a = new Money (2.51, "USD"); 
Money d2 51b = new Money (2.51, "DEM") 
assert(! d2 51a.equals(d2 51b)); 


Erehe soe ne Tenn din 
[ roce lianas Tete po pee] 


Fragen 





Mm 9 meo m tee U 
Eres wd f amne 


e 


[ Faut 0.23 onda PII 
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Testing HashCode 








MoneyTester 
public void testHash() { 

Money d2 51a = new Money (2.51, "USD") 

Money d2 51b = new Money (2.51, "USD") 

assertEquals (d2 51a.hashCode(), d2 51b.hashCode()); 
} 


Money 
public int hashCode() { 
return currencyCode.hashCode() ^ 
(int) _amountI nPenni es 


Enter veo we Term tae 


[rocwy taney uites ene [Dr] 





Muw 4 ees owe 8 
nor wd famae 


Cs 


[Frana iran oe 
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The addition method 











public Money plus (Money arg) { 


return new Money ( 
_amountinPennies + arg. amountinPenni es, 
_currencyCode); 


Eréw Fe aie CN TEM Care a 
[erue Mm mn 


Pues 


Fuse 7 Gero © Fist 14 
Earn sacf abate 


we Tracy ef ers toda a capi T N ame TA OTE 
rg 


[ransizmexxm —————— ei 
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Addition with a marked 
constructor 











public Money plus (Money arg) { 


} 


return new Money ( 
_amountI nPennies + arg._amountI nPenni es, 
_CurrencyCode, 
false); 


private Money (long amountI nPennies, String currencyCode, 


{ 


} 


boolean privacyMarker ) 


_amountinPennies = amount! nPennies; 
_currencyCode = currencyCode; 


ati ra Cava ote Toana sas 
ine mew |" ] 
feat 
wax Po wo a Fan 
Cr oat ane 
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Adding different currencies 











C] For this application, we will treat this as 
an error 


> An alternative is the MoneyBag pattern 


public void testAdditionOfDifferentCurrencies() { 
Money d15 = new Money (15, "USD") 
Money m2 51 = new Money (2.51, "DEM") 
try { 
dl5.plus(m2 51); 
assert (false) 
) catch (Illegal ArgumentException e) {} 
} 
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The new plus method 








public Money plus (Money arg) { 
if (! currencyCode.equals(arg. currencyCode) 
throw new Illegal ArgumentException 
("Cannot add different currencies"); 
return new Money 
.amounti nPennies + arg. amountinPenni es 
_currencyCode, false) 
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Duplication of test setup code 








public void testAdditionOfDifferentCurrencies() { 


Money d15 
Money m2 5 
try { 
d15. plus 
assert ( 
) catch ( 


= new Money (15, "USD"); 
= new Money (2.51, "DEM") 


(m? 51) 
alse); 
legal Argument Exception e) {} 


public void testAddition() { 


Money d15 
Money d2 5 
assertEqua 


} 
public void t 


Money d2_5 
Money d2_5 
assert(! d 


} 


public class 


= new Money (15, "USD"); 
= new Money (2.51, "USD") 
s (new Money (17.51, "USD"), dl5.plus(d2 51)); 





stDifferentCurrencyNotEqual() { 
new Money (2.51, "USD") 

new Money (2.51, "DEM') 

2 5la.equals(d2 51b)); 








e 
à 
b 
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Create a test fixture 











MoneyTester extends TestCase{ 


private Money d15; 
private Money d2 51; 
private Money m2 51; 


public void s 


etUp() { 


d15 = new Money (15, "USD"); 
d2 51 = new Money (2.51, "USD"); 
m2 51 = new Money (2.51, 'DEM'J; 


} 


public void t 


estDifferentCurrencyNotEqual() { 


assert(! d2_51.equals(m2_51)); 


} 
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Adding Subtraction 








MoneyTester 

public void testSubtraction() { 
assertEquals (new Money (12.49, "USD"), 

d15. mi nus(d2_51)); 

} 


Money 
public Money minus (Money arg) { 
if (! currencyCode.equals(arg. currencyCode) 
throw new Illegal ArgumentException ("Cannot add 
different currencies"); 
return new Money ( amountinPenni es 
arg. amounti nPennies, _currencyCode, false) 


} 
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Duplicate Code 








public Money minus (Money arg) { 
if (! currencyCode.equals(arg. currencyCode) 
throw new Illegal ArgumentException ("Cannot add 
different currencies"); 
return new Money ( amountI nPenni es 
arg. amounti nPennies, currencyCode, false) 
} 
public Money plus (Money arg) { 
if (! _currencyCode. equal s(arg. currencyCode) 
throw new Illegal ArgumentException ("Cannot add 
different currencies"); 
return new Money (_amountI nPennies + 
arg. amounti nPennies, currencyCode, false) 


} 


C] Kill such snakes immediately 
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Extract Methods 











public Money minus (Money arg) { 
assertSameCurrency(arg) 
return new Money ( amountl nPenni es 

arg. amounti nPennies, currencyCode, false) 


} 


public Money plus (Money arg) { 
assertSameCurrency(arg) 
return new Money (_amountInPennies + 

arg. amounti nPennies, currencyCode, false) 


} 


public void assertSameCurrency (Money arg) { 
if (! currencyCode.equals(arg. currencyCode) 
throw new Illegal ArgumentException ("Currencies must 
be the same") 


} 


C) Make it work, make it right 
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Next Step: Local Printing 











Leave other arithmetic and sort 
operations to the reader 


Provide a localString method that formats 
the currency in the native locale of the 
currency 


public void xtestLocalPrinting() { 
llassertEquals("*$15.00", d15. local String()); 
llassertEquals('*2,51 DM", m2_51.local String()) 
} 





Q We need a currency class 
> refactor the money class to use a currency: 


Q Define the test, but don’t run it yet 
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— Replace Data Value with Object 











You have a data item that needs additional data or behavior 
Turn the data item into an object 





Order 





customer: String 

















Customer 





name: String 
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Replace Data Value with 
Money Object 











public Money(double amount, String currencyCode) { 
.amounti nPennies = Math.round (amount * 100); 
currency = new Currency(currencyCode); 


public boolean equals (Object arg) { 
if (! (arg instanceof Money)) return false; 
Money moneyArg = (Money) arg; 
return ( amountinPennies == moneyArg. amountinPennies && 


_Currency. get Code().equals(moneyArg. currency.getCode())); 
} 

private long amountl nPennies; 

private Currency currency; 


Currency 
public Currency(String code) ( 
_code = code; 


} 
public String getCode() { 
return code; 


} 


private String _code; 
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Code in the wrong place 








Money 
public boolean equals (Object arg) ( 
if (! (arg instanceof Money)) return false; 
Money moneyArg = (Money) arg; 
return ( amountinPennies == moneyArg. amountinPennies && 


.currency.getCode().equals(moneyArg. currency.getCode())); 
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Move Method 











Money 
public boolean equals (Object arg) { 
if (! (arg instanceof Money)) return false; 
Money moneyArg = (Money) arg; 
return ( amountinPennies == moneyArg. amountinPennies && 
_Currency. equal s(moneyArg. currency)); 
} 


Currency 

public boolean equals( Object arg) { 
if (! (arg instanceof Currency)) return false; 
Currency currencyArg = (Currency) arg; 
return ( code.equals(currencyArg. code)); 


} 
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Currency is a value 

















Currency 




















Money 

public Money(double amount, String currencyCode) { 
.amounti nPennies = Math.round (amount * 100); 
currency = new Currency(currencyCode); 
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Replace Constructor with 
Factory Method 











You want to do more than simple construction when you 
create an object 
Replace the constructor with a factory method 


Employee (int type) { 
.lype = type; 
} 


static Employee create(int type) ( 
return new Empl oyee(type]; 


} 
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Replacing the Constructor 








class Currency... 

public static Currency create (String code) { 
return new Currency (code) 

} 

private Currency(String code) { 
_code = code; 


} 


class Money... 

public Money(double amount, String currencyCode) { 
.amounti nPennies = Math.round (amount * 100) 
currency = Currency.create(currencyCode) 


} 
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Replace value object 
with reference object 











class Currency... 
private String _code; 
private static Dictionary instances = new Hashtabl e() 


public static void loadinstances() { 
_instances.put("USD", new Currency("USD")); 
_instances.put("GBP", new Currency("GBP")); 
_instances.put("DEM", new Currency("DEM") ) 
} 


public static Currency create (String code) { 
Currency result = (Currency) _instances. get(code) 
if (result == null 
throw new Illegal ArgumentException 
("There is no currency with code: " + code) 
return result 
} 
class MoneyTester.. 
public void setUp() { 
Currency.loadl nstances(); 
d15 = new Money (15, "USD"); 
d2 51 new Money (2.51, "USD"); 
m2 51 new Money (2.51, "DEM') 
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Add the locale to currency 





class Currency... 

private Currency(String code, Locale locale) { 
_code = code; 
_locale = locale; 

} 

public static void loadinstances() { 
_instances.put("USD", new Currency("USD", Locale. US)); 
_instances.put("GBP", new Currency("GBP", Locale. UK)); 


_instances.put("DEM', new Currency("DEM', 
Local e. GERMANY) ) ; 


} 


private Locale _locale; 


Page 105 © Martin Fowler 9/11/99 








Add methods for printing 








class Money... 
public String local String() { 

return currency. getFormat().format(getAmount()); 
} 


class Currency... 
public NumberFormat getFormat() { 

return NumberFormat.getCurrencyl nstance( locale); 
} 


C] Enable the test 


Kl Aus Test Suse 

Enbe he nomme of he Tee ae cms 

[mre T [L5] 
Progress: 

| 





Rue 10 Bree © lue © 
Grove and Farar 





Rr 


[ Fristad 0 41 sconto Gut 
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— The Rhythm of Development 











Define a test 


Refactor to make it easy to add the 
function 


Add functionality 

Enable the test 

Refactor to remove any bad smells 
Integrate 
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Daily Build 








C] Build system every day 
> compile, link, and unit tests at 100% 
> Anyone who breaks build must fix it 
immediately 
C) Developers should check in daily 
> If more than 2 days - raise flag 


> break down coding effort for intermediate 
build 

> developers do personal build before checking 
in 


Q Assign a build token 
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Bad Smells in Code 











“If it stinks, change it.” 
— Grandma Beck, discussing child raising philosophy 


Q How do we know when to refactor 
Q No hard and fast rules 


Q Bad Smells are things to look for 
> suggest certain refactorings 
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5! 





Duplicated Code 











Same expression in two methods of the 
same class 


> Use Extract Method 
Same expression in sibling subclasses 
> Extract Method and Pull Up Method 
Similar code in sibling subclasses 
> Use Form Template Method 
Same code in unrelated classes 


2 Decide where it should really be and use 
Move Method to get it there. 


> May be a signal for Extract Class 
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Pull Up Method 








You have methods with identical results on subclasses 
Move them to the superclass 








Employee 
getName 


Salesman Engineer 





Employee 





















































Salesman Engineer 
































getName getName 
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Extract Class 











You have a class that is doing the work that should be done 
by two. 
Create a new class and move the relevant fields and 
methods from the old class into the new class. 





Person 





name 
officeAreaC ode 
officeN umber 











getT elephoneNumber 


Ų 


officeTelephone 


1 











Telephone Number 








areaCode 
number 





name 








getTelephoneNumber 














getTelephoneNumber 
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Long Method 











Q Use Extract Method on logical chunks 
> For conditions: Decompose Conditional 
Q Lots of temps make extraction difficult 


> Use Replace Temp with Query 


> For parameters use Introduce Parameter 
O bject and Preserve Whole O bject 


> Asa last resort use Replace Method with 
Method Object 
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Decompose Conditional 








You have a complicated conditional (if-then-else) statement 
Extract methods from the condition, then part, and else 
parts. 


if (date.before (SUMMER START) || date.after(SUMMER END]) 
charge = quantity * winterRate + winterServiceCharge; 
else charge = quantity * summerRate; 


if (notSummer (date]] 
charge = wi nterCharge( quantity); 
else charge = summerCharge (quantity); 
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Replace Method with Method Object 








You have a long method that uses local variables in such a 
way that you cannot apply Extract Method 
Turn the method into its own object. 


class Order... 
double price() { 
double pri maryBasePrice; 
double secondaryBasePrice; 
double tertiaryBasePrice; 
1| long computation; 








PriceCalculator 








primaryBaseP rice 
- secondaryBaseP rice 
price() Q tertiaryBaseP rice 








compute 


























retum new PriceCalculator(this).compute() 
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Preserve Whole Object 











You are getting several values from an object and passing 
these values as parameters in a method call 
Send the whole object instead 


int low = daysTempRange().getLow(); 
int high = daysTempRange().getHigh(); 
withinPlan = plan. withinRange(low, high); 


withinPlan = plan. withinRange(daysTempRange()); 
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Introduce Parameter Object 








You have a group of parameters that naturally go together 
Replace them with an object 





Customer 








amountinvoicedln(start: Date, end: Date) 
amountR eceivedin(start: Date, end: Date) 
amountOverdueln(start: Date, end: Date) 














Customer 








amountinvoicedIn(DateR ange) 
amountR eceivedin(DateR ange) 
amountO verdueln(DateR ange) 
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Large Class 











ū Find a bunch of data and methods that go 
together 
> Use Extract Class or Extract Subclass 
C) Examine how clients use the class 


> separate different kinds of uses with Extract 
Interface 


C) Complex GUI Classes 
> Use Extract Class to create domain objects. 


> Use Duplicate O bserved Data where data 
needs to be in both places 
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Duplicate Observed Data 








You have domain data available only in a gui control and 
domain methods need access. 
Copy the data to a domain object. Set up an observer to 
synchronize the two pieces of data. 





Interval Window 





startF ield: TextField 
endF ield: TextField 


lengthF ield: TextField «interface» 
dd i Observer 











Interval Window StartF ield_FocusLost 





EndField_FocusLost 


startF ield: TextField LengthField FocusLost 


endF ield: TextField 
lengthField: TextField 











StartField FocusLost 1 
EndField FocusLost 
LengthField FocusLost 








calculateLength Interval 


calculateE nd 





start: String 
end: String 








length: String Observable 








calculateLength 
calculateE nd 
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6l 








Extract Interface 





Several clients use the same subset of a class's interface or 
two classes have part of their interfaces in common 
Extract that subset into an interface 





Employee 








getRate 
hasSpecialS kill 
getName 
getDepartment 
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«interface» 
Billable 











getRate 
hasSpecials kill 











Employee 











getRate 
hasSpecialS kill 
getName 
getDepartment 
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Extract Subclass 





A class has features that are only used by some instances 
Create a subclass for that subset of features 





Job Item 








getTotalP rice 
getUnitP rice 
getE mployee 
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> 











Job Item 








getTotalP rice 
getUnitP rice 

















Labor Item 








getUnitP rice 
getE mployee 














© Martin Fowler 9/11/99 








Long Parameter Lists 








C] Parameters that seem to go together 
> Preserve Whole O bject 
> Introduce Parameter O bject 
Q) The invoked method can find one 
parameter itself 
> Use Replace Parameter With Method 


Page 123 © Martin Fowler 9/11/99 








Replace Parameter With Method 








An object invokes a method, then passes the result as a 
parameter for a method. The receiver could also invoke this 
method. 

Remove the parameter and let the receiver invoke the 
method 


int basePrice = quantity * itemPrice; 
discountLevel = getDiscountLevel(); 
double finalPrice = discountedPrice (basePrice, discountLevel }; 


V 


int basePrice = quantity * itemPrice; 
double finalPrice = discountedPrice (basePrice); 
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Divergent Change 








Q Change one class in different ways for 
different reasons 


Q Usually only apparent during evolution of 
a system 


> Use Extract Class to factor out each style of 
change 
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Shotgun Surgery 


Q A Common kind of change affects several 
classes 


Q Need to bring the changes together to 
make change easier 


> Move Method and Move Field to bring 
common elements together 


> Inline Class to remove unnecessary 
separations 
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Inline Class 








A class isn't doing very much 
Move all its features into another class and delete it. 








Telephone Number 





officeTelephone 





areaCode 
number 





name 





1 


V 


Person 





getTelephoneNumber 














getTelephoneNumber 














name 
areaCode 
number 











getTelephoneNumber 
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Feature Envy 











C) A method uses more features from 
another class than it does its own class 
> Use Move Method to move it to where it 
wants to be 
> If only part of a method is jealous use Extract 
Method and Move Method 
(C) Many patterns deliberately break this rule 


> To avoid smells of Divergent Change or 
Shotgun Surgery 
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6: 








Data Clumps 








C] Data Items that tend to hang around 
together 
> Groups of fields in several classes 
> Groups of parameters in several method calls 
> eg: startDate and endDate 
Q) Start with classes 
> Use Extract Class to group fields into an 
object 
Q) Continue with method calls 
> Preserve Whole O bject 
> Introduce Parameter O bject 
Q Atest: if you delete on data item, do the 
others make sense? 
Q Now look for methods on other classes 


that have Feature Envy for the new 
classes 
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Primitive Obsession 











Q Objects blur the line between primitive 
data types and records 

Q Objects are almost always more useful 
Replace Data Value with O bject 
Replace Type Code with Class 
Replace Type Code with Subclasses 
Replace Type Code with State/Strategy 
Replace Array with O bject 


ū Look for Data Clumps 








Page 130 © Martin Fowler 9/11/99 





— Replace Data Value with Object 











You have a data item that needs additional data or behavior 
Turn the data item into an object 





Order 











customer: String 











Customer 

















name: String 
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Change Value to Reference 











You have a class with many equal instances that you want to 
replace with a single object 
Turn the object into a reference object 








Customer 

















name: String 














Customer 

















name: String 
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— Replace Type Code with Class 











A class has a numeric type code that does not affect its 
behavior 
Replace the number with a new class 














1 














oodGroup zint BloodGroup 








O: BloodGroup 
A : BloodGroup 
B : BloodGroup 
AB : BloodGroup 











Page 133 © Martin Fowler 9/11/99 





— Replace Type Code with Subclasses 











You have an immutable type code which affects the 
behavior of a class 
Replace the type code with subclasses 








Employee 

















Employee 
ENGINEER : int — 
SALESMAN : int 


type : int 


























Engineer Salesman 
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Replace Array with Object 











You have an array where certain elements mean different 
things 
Replace the array with an object, with a field for each 
element 


String[] row = new String[3]; 
row [0] = "Liverpool"; 
row [1] = '15*; 


V 


Performance row = new Performance(); 
row. setName("Liverpool"); 
row. set Wins("15"); 
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Switch Statements 











Q Usually leads to duplicated conditionals 
> particularly when used with a type code 

Q Setup structure and use polymorphism 
> Extract Method to remove the conditional 
> Move Method to put it in the right place 
> Replace Type Code with Subclasses 
> Replace Type Code with State/Strategy 
> Replace Conditional with Polymorphism 

ū Conditional behavior on a parameter 


> Consider Replace Parameter with Explicit 
Methods 


> But try to remove the parameter 


Q For null tests 
> Introduce Null O bject 
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Replace Parameter with 
Explicit Methods 











You have a method with a runs different code depending on 
the values of an enumerated parameter 
Create a separate method for each value of the parameter 


void setValue (String name, int value) ( 
if (name. equal s("height"]) 
height = value; 
if (mame. equals("width")) 
_width = value; 
Assert. shouldNeverReachwere(); 


void setHeight(int arg) { 
height = arg; 


} 
void setWidth (int arg) { 


Width = arg; 
} 
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Introduce Null Object 








You have a method with a runs different code depending on 
the values of an enumerated parameter 
Create a separate method for each value of the parameter 


if (customer == null) plan = BillingPlan.basic(); 
else plan = customer. getPlan(); 





Customer 











getP lan 











Null Customer 





getP lan 
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Parallel Inheritance Hierarcies 











Coupling between separate hierarcies 


Often signalled by common prefixes or 
suffixes 


Make one hierarchy refer to the other 
Move features to the latter 
Remove the former 
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Lazy Class 








Q Aclass that does not do enough 
Q Remove it 

> Collapse Hierarchy 

> Inline Class 
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71 








Collapse Hierarchy 








A superclass and subclass are not very different 
Merge them together 





Employee 














Employee 

















Salesman 
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Speculative Generality 








Unused features that are there because 
you are "sure" you'll need them 


Unused features make the program hard 
to understand, and are usually wrong 


You can always add them later 


So remove them 
> Lazy Abstract Classes: Collapse Hierarchy 
> Unnecessary delegation: Inline Class 
> Unused parameters: Remove Parameter 


> Odd abstract method names: Rename 
Method 
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Temporary Field 











Q A field that's only used for a short part of 
a class's life 
Q Ifthere's more than one: separate them 
into their own class 
> Extract Class 


> Avoid conditionals with Introduce Null 
O bject 
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Message Chain 








a getO bject().getAnotherO bject().getYetAno 
therO bject().getYetAnotherAnotherO bject 
().somethingM eaningful() 

Q Couples host to a whole data structure 

Q Hide the structure 

> Hide Delegate 
> But may result in Middle Men 
Q See what the final code is doing 
> Use Extract Method on the code that uses it 
> Use Move Method to move it down the chain 
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Hide Delegate 











A client is calling a delegate class of an object 
Create methods on the server to hide the delegate 








Client Class Client Class 




















Person 














Department getManager 











getDepartment getManager 




















Department 
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Middle Man 











Q Chasing around a lot of empty delegation 


Q Talk to the real object 
Remove Middle Man 
But beware of Message Chains 


If several methods use the same delegation: 
Inline Method 


Replace Delegation with Inheritance 
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— Replace Delegation with Inheritance 








You're using delegation and are often writing many simple 


delegations for whole interface 


Make the delegating class a subclass of the delegate 








Employee 





Person Person 




















getNameo 


getName getName 
































return person.getName() 

















Employee 
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Inappropriate Intimacy 








Q Classes should not know too much about 
each other 


Q Break up classes to reduce needed links 


Use Move Method and Move Field to 
separate pieces 


Change Bidirectional Association to 
Unidirectional 


Extract Class to combine common interests 
Hide Delegate to let another class mediate. 


C] Inheritance often increases coupling 


> 
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Replace Inheritance with Delegation 
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7 





— Change Bidirectional Association 
to Unidirectional 











You have a two way association but one class no longer 
needs features from the other. 
Drop the unneeded end of the association 








Customer 





























Customer 
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— Replace Inheritance with Delegation 











A subclasses only uses part of a superclasses interface, or 
does not want to inherit its data. 
Create a field for the superclass, adjust methods to delegate 
to the superclass, remove the subclassing 











Vector 
Stack Vector 

















isEmpty 























isEmpty o isEmpty 























return. vector.isE mpty() 
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Alternative classes with 
different interfaces 











Q Try to ensure classes use different 
implementations with the same interface 
Rename Method to get names the same 


Move Method if one class does not do 
enough 


Extract Superclass to factor out 
commonalities 


Extract Interface if you can't superclass 
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Extract Superclass 











You have two classes with similar features 
Create a superclass and move the common features to the 
superclass 








Department Party 














getTotalAnnualC ost getAnnualCosi 
getName getName 


getHeadCount 
getAnnualCost 
getName getAnnualCos getAnnualCos 
getid getid getHeadCoun 























Employee 











Employee Department 
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Incomplete Library Class 








Q Cannot change library classes 


Q So usual tactics don't work 
> Introduce Foreign Method 
> Introduce Local Extension 
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Introduce Foreign Method 








A server class you are using needs an additional method, 
but you can't modify the class. 

Create a method in the client class with an instance of the 
server class as its first argument 


Date newStart = new Date (previousEnd. getYear(), 
previousEnd.getMonth(), previousEnd.getDate() * 1); 


V 


Date newStart = nextDay(previ ousEnd) ; 


private static Date nextDay(Date arg) ( 
return new Date (arg. getYear(),arg.getMonth(], arg.getDate() + 1); 


} 
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Introduce Local Extension 








A server class you are using needs several additional 
methods, but you can't modify the class. 
Create a new class which contains these extra methods. 
Make this extension class a subclass or a wrapper of the 
original 
































Client Class 
nextDay(Date) : Date = MfDate 














nextDay() : Date 











Page 155 © Martin Fowler 9/11/99 





Data Class 











Q Aclass that is just getters and setters 


Q May have public data 
> Encapsulate Field 
> Encapsulate Collection 
> Remove Setting Method 
Q Look for methods that use the accessors 


> Use Extract Method and Move Method to 
move behavior into the data class 


> Look to Hide Method on the accessors 





Page 156 © Martin Fowler 9/11/99 








Encapsulate Field 











There is a public field 
Make it private and provide accessors 


public String name 


private String name; 
public String getName() {return _name; } 
public void setName(String arg) {_name = arg;} 
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Encapsulate Collection 








A method returns a collection 
Make it return a read only view and provide add/remove 
methods 








Person 
Person 


tC ourses():Unmodifiable Set 
etCourses():Set E ge 
Wunde addCourse(:Course) 


removeCourse(:Course) 
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Remove Setting Method 








A field should be set at creation time and never altered 
Remove any Setting Method for that field 





Employee 











setimmutableValue 


=> 
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Employee 
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Hide Method 








A Method is not used by any other class 
Make the Method private 





Employee 








+aMethod 





=> 
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Employee 











- aMethod 
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8l 





Refused Bequest 











Q Only using some of the features of the 
parent 


Q Asign of an incorrect hierarchy 
Q Create a new sibling class 


> Push Down Method 
> Push Down Field 


() Doesn't want to support parent interface 
> Replace Inheritance with Delegation 
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Comments 











Q Not a bad smell: but is a deodorant 

Q Look for the smell that the comment is 
trying to mask 

Q Remove the smell, see if you still need 
the comment 
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Final Thoughts 








The one benefit of objects is that they 
make it easier to change. 


Refactoring allows you to improve the 
design after the code is written 

Up front design is still important, but not 
so critical 

Refactoring is an immature subject: not 
much written and very few tools 





Make it run, 
make it right, 
make it fast 
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